Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore(build): remove useless artifacts #54

Merged
merged 1 commit into from
Nov 7, 2022
Merged

chore(build): remove useless artifacts #54

merged 1 commit into from
Nov 7, 2022

Conversation

ghoullier
Copy link
Contributor

@ghoullier ghoullier commented Oct 27, 2022 •
edited
Loading

🎯 Goal

Repository script inherit those from https://github.com/actions/typescript-action template, some of them can be optimized and others dropped.

🧠 Approach

This PR:

  • add npm-run-all package to run scripts
  • drop npm run build script
  • remove dist/*.js intermediary transpiled files

🧪 Test

  • run action action targeting feat/add-parallel-scripts branch

@ghoullier ghoullier force-pushed the feat/add-parallel-scripts branch 2 times, most recently from 1067857 to 1f9e5e9 Compare November 2, 2022 16:02
@ghoullier ghoullier self-assigned this Nov 2, 2022
@ghoullier ghoullier added the enhancement New feature or request label Nov 2, 2022
@ghoullier ghoullier marked this pull request as ready for review November 2, 2022 16:11
@ghoullier ghoullier merged commit 195f6d3 into main Nov 7, 2022
@ghoullier ghoullier deleted the feat/add-parallel-scripts branch November 7, 2022 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdemangeon jdemangeon jdemangeon approved these changes

Assignees

@ghoullier ghoullier

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ghoullier @jdemangeon